New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make outlined function arguments non-aliasing #1006

Draft

newling wants to merge 1 commit into nod-ai:main from newling:noalias

+17 −4

Contributor

newling commented Jan 3, 2025

Analysing the .ll and .opt.ll files with and without aliasing I think this might be reason outlining causes a slow down

Risky just to plonk this attribute on here...

Contributor Author

newling commented Jan 7, 2025

Some tests fail numerically. Created this issue to understand problem better Xilinx/llvm-aie#250


          update

newling force-pushed the noalias branch from 38dd0e7 to 0857589 Compare

January 8, 2025 23:20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

MaheshRavishankar Awaiting requested review from MaheshRavishankar MaheshRavishankar will be requested when the pull request is marked ready for review MaheshRavishankar is a code owner

nirvedhmeshram Awaiting requested review from nirvedhmeshram nirvedhmeshram will be requested when the pull request is marked ready for review nirvedhmeshram is a code owner

yzhang93 Awaiting requested review from yzhang93 yzhang93 will be requested when the pull request is marked ready for review yzhang93 is a code owner

Abhishek-Varma Awaiting requested review from Abhishek-Varma Abhishek-Varma will be requested when the pull request is marked ready for review Abhishek-Varma is a code owner

jtuyls Awaiting requested review from jtuyls jtuyls will be requested when the pull request is marked ready for review jtuyls is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet